Skip to content

Adding completion for https://github.com/swissquote/carnotzet#86

Open
matthyx wants to merge 1 commit into
juven:masterfrom
matthyx:master
Open

Adding completion for https://github.com/swissquote/carnotzet#86
matthyx wants to merge 1 commit into
juven:masterfrom
matthyx:master

Conversation

@matthyx

@matthyx matthyx commented Oct 27, 2017

Copy link
Copy Markdown

No description provided.

@mosabua

mosabua commented Apr 21, 2020

Copy link
Copy Markdown
Collaborator

This goes beyond the scope of this project imho. We can take the completion of the maven plugin only if you change the PR.

@matthyx

matthyx commented Apr 21, 2020

Copy link
Copy Markdown
Author

Alright, so I just skip the docker-compose part? And fix the conflict...

Comment thread bash_completion.bash Outdated
local plugin_goals_spring_boot="spring-boot:run|spring-boot:repackage"
local plugin_goals_jgitflow="jgitflow:feature-start|jgitflow:feature-finish|jgitflow:release-start|jgitflow:release-finish|jgitflow:hotfix-start|jgitflow:hotfix-finish|jgitflow:build-number"
local plugin_goals_wildfly="wildfly:add-resource|wildfly:deploy|wildfly:deploy-only|wildfly:deploy-artifact|wildfly:redeploy|wildfly:redeploy-only|wildfly:undeploy|wildfly:undeploy-artifact|wildfly:run|wildfly:start|wildfly:shutdown|wildfly:execute-commands"
local plugin_goals_zet="addrs|clean|crawl|help|logs|ps|pull|restart|run|shell|start|stop|welcome"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just have this specific line but add the right prefix..

Comment thread bash_completion.bash Outdated
local IFS=$'|\n'

if [[ ${cur} == -D* ]] ; then
if [[ ${words[1]} == zet:* ]] ; then

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove all that

Comment thread bash_completion.bash Outdated
POM_HIERARCHY=()
__pom_hierarchy
_get_comp_words_by_ref -n : cur prev
_get_comp_words_by_ref -n := cur prev words

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove that change

Comment thread bash_completion.bash Outdated
fi
}

__docker_compose_services()

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove that change

@mosabua

mosabua commented Apr 21, 2020

Copy link
Copy Markdown
Collaborator

Might be worth just creating a new PR if you dont want to muck around with rebasing..

@matthyx

matthyx commented Apr 21, 2020

Copy link
Copy Markdown
Author

@mosabua like that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants